Skip to content

Conversation

@BenOsodrac
Copy link
Contributor

@BenOsodrac BenOsodrac commented Feb 11, 2025

Issue number: internal


What is the current behavior?

What is the new behavior?

Added new utility-classes related to:

  • flex
  • position
  • size

Does this introduce a breaking change?

  • Yes
  • No

Other information

@BenOsodrac BenOsodrac added type: feature request a new feature, enhancement, or improvement package: core @ionic/core package labels Feb 11, 2025
@vercel
Copy link

vercel bot commented Feb 11, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ionic-framework ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 11, 2025 5:02pm

@BenOsodrac BenOsodrac marked this pull request as ready for review February 11, 2025 17:33
@BenOsodrac BenOsodrac requested a review from a team as a code owner February 11, 2025 17:33
@BenOsodrac BenOsodrac requested a review from thetaPC February 11, 2025 17:33
Copy link
Contributor

@thetaPC thetaPC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since these appear to be more directed towards the ionic theme, it might be best to move them to the css/ionic folder. Why are they being added to core?

@BenOsodrac
Copy link
Contributor Author

Since these appear to be more directed towards the ionic theme, it might be best to move them to the css/ionic folder. Why are they being added to core?

The request come from the Ionic theme context yes, but I its all classes related to agnostic/normal css rules, that I see no problem adding to the common context, as some of them, like the flex ones, are just an extension of the existing offering.

Copy link
Contributor

@thetaPC thetaPC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@BenOsodrac BenOsodrac merged commit 7be7c08 into next Feb 12, 2025
48 checks passed
@BenOsodrac BenOsodrac deleted the ROU-11465 branch February 12, 2025 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: core @ionic/core package type: feature request a new feature, enhancement, or improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants